Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support Pandas 2.2, unpin Pyarrow, and fail on warnings in tests #143

Merged
merged 15 commits into from
Aug 13, 2024
Merged

Conversation

hoxbro
Copy link
Member

@hoxbro hoxbro commented Aug 7, 2024

Fixes #137

The pandas 2.2 compatibility changes are almost the same as those done in GeoPandas:

The dask warning was added in dask 2024.1:

Removed legacy pyarrow support as we now lower pin to version 10

I have also added HYPOTHESIS_MAX_EXAMPLES to make it easier to test locally.

Added packaging as a required dependency.

@hoxbro hoxbro changed the title Unpin Pandas 2.2 and fail on warnings in tests Unpin Pandas, Pyarrow, and fail on warnings in tests Aug 7, 2024
@hoxbro hoxbro marked this pull request as draft August 7, 2024 18:37
Base automatically changed from pixi to main August 7, 2024 20:18
Copy link

codecov bot commented Aug 8, 2024

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@hoxbro hoxbro marked this pull request as ready for review August 8, 2024 10:33
@hoxbro hoxbro changed the title Unpin Pandas, Pyarrow, and fail on warnings in tests Support Pandas 2.2, unpin Pyarrow, and fail on warnings in tests Aug 8, 2024
@hoxbro hoxbro requested a review from philippjfr August 8, 2024 10:48
spatialpandas/geoseries.py Outdated Show resolved Hide resolved
spatialpandas/geoseries.py Outdated Show resolved Hide resolved
Copy link
Member

@philippjfr philippjfr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure I have the context to give this the review it deserves but fine with this for now.

@hoxbro hoxbro merged commit e7798e3 into main Aug 13, 2024
11 checks passed
@hoxbro hoxbro deleted the unpins branch August 13, 2024 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Pandas 2.2.0 rc0 emit warning
2 participants